-
Notifications
You must be signed in to change notification settings - Fork 16
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am sincerely sorry for taking so long to review this. This looks good to me, just a couple of tweaks please.
If you can make these changes and update your branch I am happy to merge and include in a release next week.
README.md
Outdated
"status": "429", | ||
} | ||
}) | ||
{'results': [], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you update the formatting here to match the formatting of other responses in this README?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You have a great point. How about "pretty"?
Thanks again for adding this. I'm going to merge and include in a patch release, 7.8.1, shortly. |
No description provided.